Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Steam transport #144

Closed
wants to merge 1 commit into from
Closed

Conversation

UkoeHB
Copy link
Contributor

@UkoeHB UkoeHB commented Feb 3, 2024

This PR removes the Steam transport.

Transports other then netcode and the in-memory transport should be supported by third-party crates (Steam and WebTransport primarily). Renet has been trapped by the steamworks-rs dependency for months, and this can happen again in the future. This crate should provide and maintain the core transport-agnostic API plus the original netcode implementation. It should not be responsible for maintaining other transports.

Until someone willing to maintain the Steam transport can be found, I recommend adding a dev branch to this repo with the existing Steam code.

@lucaspoffo
Copy link
Owner

You are right,

I decided to move the steam support for bevy to the renet_steam crate enabled by a feature instead of removing it. This would keep the steam development separeted from the bevy plugin. Done in 9584c69

@lucaspoffo lucaspoffo closed this Feb 8, 2024
@UkoeHB UkoeHB deleted the remove_steam branch February 16, 2024 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants